workspace: Do window slot computing in three steps

This ensures that we have the correct Y value when sorting
windows.

https://bugzilla.gnome.org/show_bug.cgi?id=694469

diff --git a/js/ui/workspace.js b/js/ui/workspace.js
index e8604b49f34a1e6b1d994536ef02d123505f3c09..145411b80942d63c8b67f3a477ea3569f630cc18 100644 (file)
--- a/js/ui/workspace.js
+++ b/js/ui/workspace.js
@@ -767,23 +767,30 @@ const LayoutStrategy = new Lang.Class({
 
         let slots = [];
 
+        // Do this in three parts.
+        let height = 0;
+        for (let i = 0; i < rows.length; i++) {
+            let row = rows[i];
+            height += row.height + this._rowSpacing;
+        }
+
+        height -= this._rowSpacing;
+
         let y = 0;
+
         for (let i = 0; i < rows.length; i++) {
             let row = rows[i];
             row.x = area.x + (area.width - row.width) / 2;
-            row.y = area.y + y;
+            row.y = area.y + y + (area.height - height) / 2;
             y += row.height + this._rowSpacing;
+
             row.windows.sort(Lang.bind(this, function(a, b) {
                 return this._getDistance(row, a.realWindow) - this._getDistance(row, b.realWindow);
             }));
         }
 
-        let height = y - this._rowSpacing;
-        let baseY = (area.height - height) / 2;
-
         for (let i = 0; i < rows.length; i++) {
             let row = rows[i];
-            row.y += baseY;
             let x = row.x;
             for (let j = 0; j < row.windows.length; j++) {
                 let window = row.windows[j];